Skip to content

TokenEscrow Support (XLS-85)#167

Merged
GuillemGarciaDev merged 24 commits intoXRPLF:v0.1.xfrom
florent-uzio:feat/token-escrow
Oct 17, 2025
Merged

TokenEscrow Support (XLS-85)#167
GuillemGarciaDev merged 24 commits intoXRPLF:v0.1.xfrom
florent-uzio:feat/token-escrow

Conversation

@florent-uzio
Copy link

@florent-uzio florent-uzio commented Sep 26, 2025

TokenEscrow

Description

This PR aims to:

  • Add support for the TokenEscrow amendment (XLS-85).
  • Fix the Flatten function for the Destination and Owner fields in the Escrow transaction types.
  • Adds example of the TokenEscrow with an IOU.
  • Adds a method to get the Sequence from the TxResponse which makes parsing easier for Escrows than response["Sequence"].

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Refactoring

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code where needed
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective
  • New and existing unit tests pass locally with my changes

Changes

See description

Notes (optional)

Examples working:

Screenshot 2025-09-26 at 22 17 41

@florent-uzio florent-uzio changed the title updated Escrow ledger entry type TokenEscrow Support Sep 26, 2025
@florent-uzio florent-uzio changed the title TokenEscrow Support TokenEscrow Support (XLS-85) Sep 26, 2025
@GuillemGarciaDev GuillemGarciaDev self-requested a review October 15, 2025 07:07
Copy link
Collaborator

@GuillemGarciaDev GuillemGarciaDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good PR, left some comments 👍🏻

@GuillemGarciaDev GuillemGarciaDev merged commit 5f418f8 into XRPLF:v0.1.x Oct 17, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants